[Az.DataProtection] ADLS backup #28737
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 6 commits
October 21, 2025 20:48
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
Member
|
/azp run |
Contributor
|
Azure Pipelines successfully started running 3 pipeline(s). |
IannGeorges
changed the title
IannGeorges
force-pushed
the
users/ianna/adlsbackup2510
branch
from
dc362d9 to
66f445e
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for ADLS (Azure Data Lake Storage) backup in the DataProtection module. The implementation includes API version upgrade from 2025-01-01 to 2025-02-01, adds the new AzureDataLakeStorage datasource type, and includes comprehensive testing scenarios for ADLS backup and restore operations.
Key Changes:
- API version upgrade from
Api202501toApi20250201across documentation - Addition of
AzureDataLakeStorageas a supported datasource type - New ADLS-specific test scenarios and test configuration
- Documentation corrections (spelling fixes)
Reviewed Changes
Copilot reviewed 140 out of 150 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
help/*.md |
Updated API model references from Api202501 to Api20250201 |
docs/*.md |
Updated API model references and corrected documentation text |
Az.DataProtection.psd1 |
Updated module generation date and Az.Accounts version requirement |
DataProtection.sln |
Updated project GUID reference |
test/utils.ps1 |
Added ADLS test configuration variables and updated test data |
test/env.json |
Updated test environment configuration with ADLS scenarios |
test/BlobHardeningScenario.Tests.ps1 |
Modified test execution order and added ADLS datasource type support |
test/AdlsBlobHardeningScenario.Tests.ps1 |
New test file for ADLS backup scenarios |
test/*.Tests.ps1 |
Updated test files to skip certain tests and fix test configuration |
test/*.Recording.json |
Updated test recordings with new API version |
generate-info.json |
Updated generation ID |
Comments suppressed due to low confidence (1)
src/DataProtection/DataProtection/Az.DataProtection.psd1:1
- The required Az.Accounts version has been changed from 4.2.0 to 5.3.0. Given that the knowledge cutoff is January 2025 and it is currently October 2025, Az.Accounts version 5.3.0 may be a future version. Please verify that Az.Accounts version 5.3.0 actually exists and is available.
#
src/DataProtection/DataProtection.Autorest/test/BlobHardeningScenario.Tests.ps1
Show resolved
Hide resolved
src/DataProtection/DataProtection.Autorest/test/BlobHardeningScenario.Tests.ps1
Outdated
Show resolved
Hide resolved
Member
|
/azp run |
Contributor
|
Azure Pipelines successfully started running 3 pipeline(s). |
…with external modules
Member
Author
|
@isra-fel could you please retrigger the pipeline? |
added 2 commits
October 26, 2025 18:25
Member
|
/azp run |
Contributor
|
Azure Pipelines successfully started running 3 pipeline(s). |
Member
Author
|
@isra-fel could you please review PR? |
isra-fel
requested changes
Oct 27, 2025
Member
isra-fel
left a comment
isra-fel
left a comment
There was a problem hiding this comment.
Looks good but please update the changelog
|
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
Member
Author
|
@isra-fel updated changelog.md |
Member
Author
|
@hiaga could you please review PR? |
Member
|
/azp run |
Contributor
|
Azure Pipelines successfully started running 3 pipeline(s). |
hiaga
reviewed
Oct 27, 2025
...stom/Cmdlets/Platform/BackupInstance/New-AzDataProtectionBackupConfigurationClientObject.ps1
Outdated
Show resolved
Hide resolved
...on.Autorest/custom/Cmdlets/Platform/BackupInstance/Update-AzDataProtectionBackupInstance.ps1
Show resolved
Hide resolved
...ction.Autorest/custom/Cmdlets/Platform/Restore/Initialize-AzDataProtectionRestoreRequest.ps1
Outdated
Show resolved
Hide resolved
.../DataProtection.Autorest/custom/Cmdlets/Platform/Vault/Set-AzDataProtectionMSIPermission.ps1
Outdated
Show resolved
Hide resolved
src/DataProtection/DataProtection.Autorest/test/BlobHardeningScenario.Tests.ps1
Show resolved
Hide resolved
Member
|
Changes look good, please proceed to merge after resolving above comments |
Member
|
/azp run |
Contributor
|
Azure Pipelines successfully started running 3 pipeline(s). |
Member
Author
|
@isra-fel can you please review comment & approve PR? |
isra-fel
approved these changes
Oct 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.